From: Florian Müllner <fmuellner@gnome.org>
Date: Wed, 10 Oct 2012 20:07:02 +0000 (+0200)
Subject: loginDialog: Use the same focus root for dialog and ctrl-alt-tab
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~5630
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=3abfcda8b5998d10caa129c09422dd2c25874451;p=gnome-shell.git

loginDialog: Use the same focus root for dialog and ctrl-alt-tab

Adding a group to the Ctrl-Alt-Tab popup will also add it to the
focus manager. Due to that, we currently end up with two focus
groups added for the login dialog - an explicit one for the entire
dialog, and an implicit one for the main content group.
When doing keynav, we ascend in the widget hierarchy from the
currently focused actor until we find a valid focus root, so
adding a children of the dialog as focus root breaks keynav to
any actors that are not inside the main content group.
The simple fix is to use the same group in both cases.

https://bugzilla.gnome.org/show_bug.cgi?id=684730
---

diff --git a/js/gdm/loginDialog.js b/js/gdm/loginDialog.js
index 17a4ad6ec..7ea93ba6d 100644
--- a/js/gdm/loginDialog.js
+++ b/js/gdm/loginDialog.js
@@ -1211,7 +1211,7 @@ const LoginDialog = new Lang.Class({
     },
 
     _onOpened: function() {
-        Main.ctrlAltTabManager.addGroup(this.contentLayout,
+        Main.ctrlAltTabManager.addGroup(this.dialogLayout,
                                         _("Login Window"),
                                         'dialog-password',
                                         { sortGroup: CtrlAltTab.SortGroup.MIDDLE });